Skip to content

Python: Multiple new classes tests to improve code coverage#12089

Merged
moonbox3 merged 69 commits into
microsoft:mainfrom
gaudyb:main
May 16, 2025
Merged

Python: Multiple new classes tests to improve code coverage#12089
moonbox3 merged 69 commits into
microsoft:mainfrom
gaudyb:main

Conversation

@gaudyb

@gaudyb gaudyb commented May 15, 2025

Copy link
Copy Markdown
Contributor

Motivation and Context

Improve code coverage in different classes.

Description

Add unit tests for:

  • ollama_chat_completion
  • azure_cosmos_db_mongodb_store
  • chroma utils
  • redis utils
  • process_actor
  • step_actor
  • process_builder

Contribution Checklist

@gaudyb gaudyb requested a review from a team as a code owner May 15, 2025 15:42
@markwallace-microsoft markwallace-microsoft added the python Pull requests for the Python Semantic Kernel label May 15, 2025
@github-actions github-actions Bot changed the title Multiple new classes tests to improve code coverage Python: Multiple new classes tests to improve code coverage May 15, 2025
@markwallace-microsoft

markwallace-microsoft commented May 15, 2025

Copy link
Copy Markdown
Contributor

Python Unit Test Overview

Tests Skipped Failures Errors Time
3621 12 💤 0 ❌ 0 🔥 1m 44s ⏱️

@moonbox3 moonbox3 enabled auto-merge May 16, 2025 07:14
@moonbox3 moonbox3 disabled auto-merge May 16, 2025 07:14
@moonbox3 moonbox3 added this pull request to the merge queue May 16, 2025
Merged via the queue into microsoft:main with commit 50a88b2 May 16, 2025
jcruzmot-te pushed a commit to thousandeyes/aia-semantic-kernel that referenced this pull request Sep 15, 2025
…t#12089)

### Motivation and Context

Improve code coverage in different classes.

### Description

Add unit tests for:

- ollama_chat_completion
- azure_cosmos_db_mongodb_store
- chroma utils
- redis utils
- process_actor
- step_actor
- process_builder

### Contribution Checklist

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

---------

Co-authored-by: Gaudy Blanco <gaudy-microsoft@MacBook-Pro-m4-Gaudy-For-Work.local>
Co-authored-by: Evan Mattson <35585003+moonbox3@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests for the Python Semantic Kernel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants